Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: Move callbacks to ts 1X #28710

Merged
merged 20 commits into from
Apr 1, 2023
Merged

refactor: Move callbacks to ts 1X #28710

merged 20 commits into from
Apr 1, 2023

Conversation

KevLehman
Copy link
Contributor

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@codecov
Copy link

codecov bot commented Mar 30, 2023

Codecov Report

Merging #28710 (d2a83ce) into develop (de8f27d) will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28710      +/-   ##
===========================================
+ Coverage    45.21%   45.24%   +0.03%     
===========================================
  Files          738      738              
  Lines        14461    14461              
  Branches      2236     2236              
===========================================
+ Hits          6538     6543       +5     
+ Misses        7617     7613       -4     
+ Partials       306      305       -1     
Flag Coverage Δ
e2e 45.20% <ø> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@KevLehman KevLehman marked this pull request as ready for review March 30, 2023 21:16
@KevLehman KevLehman requested review from a team as code owners March 30, 2023 21:16
@KevLehman KevLehman added the stat: ready to merge PR tested and approved waiting for merge label Mar 31, 2023
@kodiakhq kodiakhq bot removed the stat: ready to merge PR tested and approved waiting for merge label Mar 31, 2023
@kodiakhq
Copy link
Contributor

kodiakhq bot commented Mar 31, 2023

This PR currently has a merge conflict. Please resolve this and then re-add the ['stat: ready to merge', 'automerge'] label.

@ggazzo ggazzo merged commit d478dcf into develop Apr 1, 2023
@ggazzo ggazzo deleted the refactor/callbacks-ts branch April 1, 2023 16:33
gabriellsh added a commit that referenced this pull request Apr 4, 2023
…avatarEmoji

* 'develop' of github.com:RocketChat/Rocket.Chat: (63 commits)
  fix: Messages jumping after reaction (#28770)
  fix: Quotes chain off by one error in quote chain limit settings (#28281)
  docs: update documentation (#28719)
  chore: /help notifying the shortcuts in multiple messages (#28765)
  refactor: remove Users from fibers 14 (#28780)
  regression(file-upload:s3): Fix missing store import (#28781)
  refactor: remove Users from fibers 13 (#28772)
  refactor: remove Users from fibers 12 (#28767)
  ci: revert Variable Setup changes (#28754)
  feat: auto link custom domain (#28501)
  refactor: `Rooms` model 4/4 (#28742)
  regression: invalid callback callAsync (#28766)
  refactor: remove Users from fibers 11 (#28740)
  refactor: Convert our Upload Code to TS (#28711)
  refactor: Move callbacks to ts 1X (#28710)
  refactor: Remove Users from fibers 10 (#28737)
  fix: message-parser links (#28735)
  refactor(client): Miscelaneous (#28732)
  refactor: `Rooms` model 3/4 (#28736)
  refactor: `Rooms` model 2X (#28715)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants